Merge branch 'master' into ospath

fix fromOsPath on windows

Same reasons as 9c9baf7c6581235643661b94f70f76f8a10ae60e

fix toOsPath on windows

After recent changes to use OsPath, test suite fails on windows with:

Exception: Uneven number of bytes: 5. This is not a Word16 bytestream.

Hopefully this fixes it. The innefficiency of the conversion is
unfortunate.

response

more OsPath conversion

Sponsored-by: Nicholas Golder-Manning

Added a comment

Added a comment

Added a comment

Added a comment

Added a comment

more OsPath conversion

Sponsored-by: Luke T. Shumaker

Merge branch 'master' of ssh://git-annex.branchable.com

Merge branch 'master' into ospath

windows build fix

and a little more bonus RawFilePath conversion

more OsPath conversion

Sponsored-by: mycroft

Merge /home/joey/tmp/git-annex into ospath

remove Read instance for AssociatedFile

This instance is not used.

correct typo

poll vote (My phone (or MP3 player))

poll vote (My phone (or MP3 player))

comment

update

update

comment

comment

Added a comment: Simple config amendment for Apache served repositories

more OsPath conversion

keyFile has a nice improvement; since a Key is a ShortByteString, it can
be converted to an OsPath without needing the copy that was done before.

Unfortunately, fileKey has to convert from a ShortByteString to a
ByteString in order to use attoparsec, and then the results get
converted back to an OsPath, so there are now 2 copies.
Maybe attoparsec will eventually get a ShortByteString API,
see https://github.com/haskell/attoparsec/issues/225

Sponsored-by: Joshua Antonishen

optimise literalOsPath

Taking a ShortByteString and using OverloadedStrings should avoid it
being converted from a String.

The reason there is no IsString instance for OsPath is presumably the
bad behavior of IsString for ByteString on unicode btw. But
literalOsPath won't be used with unicode in git-annex.

Sponsored-by: unqueued

Merge branch 'master' of ssh://git-annex.branchable.com

comment

Added a comment

Added a comment

Added a comment

thoughts

update comment

more OsPath conversion

Merge branch 'master' of ssh://git-annex.branchable.com

fix link

reporting on FTBFS

Added a comment

Added a comment

Added a comment

Some clarifications in my reproduce steps about the state of the rsync remote

rename bugs/rsyncurl_without___34____58____34___creates_local_folder_as_repo.mdwn to bugs/rsyncurl_without___34____58____34___creates_local_folder_as_remote.mdwn

Initial post

Added a comment

Added a comment

more OsPath conversion

OsPath conversion

Decent win in exportDirectories, since it operates on ShortByteString
end to end now without needing conversion. That made it worth
implementing an OsPath specific code path there.

And ExportLocation already being a ShortByteString is an good example of why
it's a good thing that OsPath uses that!

Sponsored-by: k0ld on Patreon

Merge branch 'master' into ospath

Merge branch 'master' of ssh://git-annex.branchable.com

windows build fix

more OsPath conversion

Finally reached Annex code in this conversion.

Sponsored-by: Graham Spencer

Added a comment

Merge branch 'master' into ospath

another windows build fix

more OsPath conversion

Sponsored-by: Leon Schuermann

Merge branch 'master' into ospath

more windows and osx build fixes

more OsPath conversion

Sponsored-by: Eve

Merge branch 'master' into ospath

more windows and osx build fixes

more OsPath conversion

About 1/10th done with this I think.

Merge branch 'master' into ospath

Merge branch 'master' of ssh://git-annex.branchable.com

windows and osx build fixes for recent ByteString conversion work

avoid build warning on windows

remove a few unused imports on windows

Added a comment

Added a comment

more OsPath conversion

Git.Types now uses it, as does TopFilePath, making for plenty of new
compile errors needing fixing.

Sponsored-by: Brock Spratlen

complaining about dead

Added a comment

continue conversion

Add Utility.OsString, with a special case for length.

add System.FilePath to this conversion

It seems to make sense to convert both System.Directory and
System.FilePath uses to OsPath in one go. This will generally look like
replacing RawFilePath with OsPath in type signatures, and will be driven
by the now absolutely massive pile of compile errors.

Got a few modules building in this new regime.

Sponsored-by: Jack Hill

update

start converting from System.Directory to System.OsPath

This is the start of a long road, got the first few files to compile
after this large change.

Sponsored-by: mycroft

response

Merge branch 'master' of ssh://git-annex.branchable.com

parsePOSIXTime ByteString conversion

Some easy (though tiny) speed wins.

Sponsored-by: Luke T. Shumaker on Patreon

fix reversion

af3b9cbd3647ecfb3f3f6c4008dc9e64f7d42591 lost takeFileName, breaking the
test suite

convert from readFileStrict

This removes that function, using file-io readFile' instead.

Had to deal with newline conversion, which readFileStrict does on
Windows. In a few cases, that was pretty ugly to deal with.

Sponsored-by: Kevin Mueller

fix reversion

In 793ddecd4b72a5e4746b3b426d3bca400737118b, writeSshConfig was made to
writeFile a ByteString, which lost the newline conversion on Windows.

Added linesFile to fix it. This will also be useful for other writeFile
conversions.

convert from readFileStrict

In cases where it reads lines, which needs newline conversion on
windows.

convert from readFileStrict

More of these still to convert, but need to be careful of newline
translation.

fix reversion

793ddecd4b72a5e4746b3b426d3bca400737118b made installWrapper not do
newline conversion on windows.

use file-io for readFile/writeFile/appendFile on ByteStrings

These are all straightforward, and easy small performance wins.

Sponsored-by: Nicholas Golder-Manning

RawFilePath conversion for replaceFile

Sponsored-by: Joshua Antonishen

simplify replaceFile using relatedTemplate

Now that truncateFilePath and relatedTemplate have both been optimised,
may as well use them in replaceFile, rather than the custom hack it
used.

Removed the windows-specific ifdef as well, because on Windows long
filepaths no longer really a problem, since ghc and git-annex use UNC
converted paths.

replaceFile no longer checks fileNameLengthLimit. That took a syscall,
and since we have an existing file, we know filenames of its length are
supported by the filesystem. Assuming that the withOtherTmp directory is
on the same filesystem as the file replaceFile is being called on, which
I believe it is.

Sponsored-by: Leon Schuermann

optimize truncateFilePath

Often the filepath will be all ascii, or mostly so, and this
optimisation makes a file that has an ascii suffix of sufficient length
be roundtrip converted between String and ByteString only once, rather
than once per character.

Sponsored-by: Graham Spencer

optimize relatedTemplate

break out templateAddedLength

use openTempFile from file-io

And follow-on changes.

Note that relatedTemplate was changed to operate on a RawFilePath, and
so when it counts the length, it is now the number of bytes, not the
number of code points. This will just make it truncate shorter strings
in some cases, the truncation is still unicode aware.

When not building with the OsPath flag, toOsPath . fromRawFilePath and
fromRawFilePath . fromOsPath do extra conversions back and forth between
String and ByteString. That overhead could be avoided, but that's the
non-optimised build mode, so didn't bother.

Sponsored-by: unqueued

rename forum/reupload_existing_files_to_bare_repo.mdwn to forum/reuploads_existing_files_to_bare_repo.mdwn